Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: updates route-level idle timeout ID #1206

Merged
merged 1 commit into from
Jan 29, 2024
Merged

Conversation

ZPain8464
Copy link
Contributor

@ZPain8464 ZPain8464 commented Jan 29, 2024

I know we've documented internally how we document our route- and global-level conventions, but this setting predates those decisions. For now, I've updated the route-level idle timeout setting ID to route-idle-timeout, as that was the original name given to the route-level setting. The Console and Zero repos' help docs also label this setting "route-idle-timeout."

I've filed issues on both repos to update the ID:

Fixes #1205

@ZPain8464 ZPain8464 requested a review from kenjenkins January 29, 2024 14:29
@ZPain8464 ZPain8464 requested a review from a team as a code owner January 29, 2024 14:29
@ZPain8464 ZPain8464 requested review from cmo-pomerium and removed request for a team January 29, 2024 14:29
Copy link

netlify bot commented Jan 29, 2024

Deploy Preview for pomerium-docs ready!

Name Link
🔨 Latest commit 5bd1b96
🔍 Latest deploy log https://app.netlify.com/sites/pomerium-docs/deploys/65b7b63e85e64500084d1101
😎 Deploy Preview https://deploy-preview-1206--pomerium-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ZPain8464 ZPain8464 requested a review from nhayfield January 29, 2024 14:41
Copy link
Contributor

@nhayfield nhayfield left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The ID is not used in console or zero. Only the outer key is used. No updates are needed for console and zero.

@ZPain8464 ZPain8464 merged commit 9d72c1d into main Jan 29, 2024
@ZPain8464 ZPain8464 deleted the zpain/fix-idle-timeout-id branch January 29, 2024 15:11
backport-actions-token bot pushed a commit that referenced this pull request Jan 29, 2024
updates route-level idle timeout ID
ZPain8464 added a commit that referenced this pull request Jan 29, 2024
Docs: updates route-level idle timeout ID (#1206)

updates route-level idle timeout ID

Co-authored-by: zachary painter <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Idle Timeout" fills up settings reference table when changing sort order
2 participants